Skip to content

all: generate and embed block-access-lists in block bodies when --experimentalbal is enabled #32586

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

all: generate and embed block-access-lists in block bodies when --experimentalbal is enabled #32586

wants to merge 6 commits into from
+1,681 −659

Conversation

jwasinger
Copy link
Contributor

@jwasinger jwasinger commented Sep 11, 2025 •
edited
Loading

  • Add EIP-7928 block access lists as a field on the block body, and the BAL hash as a field in the block header.
  • Introduce a BlockAccessListTracer which will generate an access list from the execution of a block.
  • introduce --experimental.bal flag which will generate block access lists on execution of all post-cancun blocks, and embed them on the block body before importing.
    • This option is mainly meant for testing the BAL implementation using the existing spec test coverage (generate the BALs during block import while running a test and then re-execute the test but using the BALs for block execution). Accordingly, this option does not set the BAL hash in the block header.

Depends on:

@jwasinger jwasinger marked this pull request as ready for review September 16, 2025 11:54
MariusVanDerWijden
MariusVanDerWijden reviewed Sep 17, 2025
View reviewed changes
MariusVanDerWijden
MariusVanDerWijden reviewed Sep 17, 2025
View reviewed changes
jwasinger
jwasinger commented Sep 29, 2025
View reviewed changes
@jwasinger jwasinger force-pushed the bal-generation branch 2 times, most recently from 8d479ef to 961f517 Compare September 29, 2025 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MariusVanDerWijden MariusVanDerWijden MariusVanDerWijden left review comments

@fjl fjl Awaiting requested review from fjl fjl is a code owner

@gballet gballet Awaiting requested review from gballet

@rjl493456442 rjl493456442 Awaiting requested review from rjl493456442 rjl493456442 is a code owner

@zsfelfoldi zsfelfoldi Awaiting requested review from zsfelfoldi

@s1na s1na Awaiting requested review from s1na s1na is a code owner

@lightclient lightclient Awaiting requested review from lightclient lightclient is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jwasinger @MariusVanDerWijden